Description

When attempting to paste a composite part into a non-composite binding both Input Manager and Input System end up throwing errors as it uses a best guess trying to find the closest applicable index it can paste into. When pasting directly into a binding nothing is found, so nothing is pasted, but attempts to return the new selection/last pasted element anyway.

This fix skips returning the new selections when nothing was actually pasted.

I attempted to look into disabling paste completely in this scenario but HasPastableClipboardData is passed the InputBinding type and I couldn't find a simple way to distinguish between composites and bindings at this point.

Testing status & QA

Tested in Input Manager and Input System (old and new UI) and confirmed that there are no longer errors in either.

Overall Product Risks

• Complexity: 0
• Halo Effect: 0

Comments to reviewers

Checklist

Before review:

• Changelog entry added.
  • Explains the change in Changed, Fixed, Added sections.
  • For API change contains an example snippet and/or migration example.
  • JIRA ticket linked, example (case %%). If it is a private issue, just add the case ID without a link.
  • Jira port for the next release set as "Resolved".
• Tests added/changed, if applicable.
  • Functional tests Area_CanDoX, Area_CanDoX_EvenIfYIsTheCase, Area_WhenIDoX_AndYHappens_ThisIsTheResult.
  • Performance tests.
  • Integration tests.
• Docs for new/changed API's.
  • Xmldoc cross references are set correctly.
  • Added explanation how the API works.
  • Usage code examples added.
  • The manual is updated, if needed.

During merge:

• Commit message for squash-merge is prefixed with one of the list:
  • NEW: ___.
  • FIX: ___.
  • DOCS: ___.
  • CHANGE: ___.
  • RELEASE: 1.1.0-preview.3.

After merge:

• Create forward/backward port if needed. If you are blocked from creating a forward port now please add a task to ISX-1444.